reverse-string: encourage grapheme awareness #261
Conversation
|
|
||
| You will probably find the `Unicode` stdlib useful for this bonus task. | ||
|
|
||
| To enable the graphemes test, add `const TEST_GRAPHEMES = true` to the global scope of your file. |
There was a problem hiding this comment.
I wish Julia had a nicer way of handling this.
There was a problem hiding this comment.
I don't think there's much that can be done. Test or build suites in other languages would read a DEFINE or config files or environment variables. We want the student to only have to modify the one file and we want mentors to run the same tests the students are running, so we probably don't want a config file or env variables.
We could define a function in runtests and give the students an API like
enable_bonus_test(:graphemes)or something?
That would require some changes to our repo-level runtests because that includes the files in a different order.
There was a problem hiding this comment.
I think how you did it works fine, it would just be nice if Julia had a nice way to run specific testsets. (there's an issue open for that I think)
Closes #238. I've put in a compulsory unicode test with an emoji and
left the grapheme tests optional for now because they're a bit fiddlier
to get right.